Skip to content

Optimize page loads and stabilize diagnostics charts - #191

Merged
bifrost0x merged 7 commits into
mainfrom
dev/optimize-initial-page-load
Sep 2, 2026
Merged

Optimize page loads and stabilize diagnostics charts#191
bifrost0x merged 7 commits into
mainfrom
dev/optimize-initial-page-load

Conversation

@bifrost0x

@bifrost0x bifrost0x commented Sep 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a dedicated static-delivery policy that applies deterministic gzip only to public CSS, JavaScript, and SVG assets, removes artificial cookie variance, and gives content-addressed assets immutable caching
  • generate a six-locale authentication-only translation bundle with a stale-output CI guard instead of loading the full workspace catalog on sign-in pages
  • defer decorative authentication backgrounds until after the load event and derive direct template/CSS asset versions from SHA-256 content hashes
  • keep diagnostics canvas dimensions stable when switching between Workspaces and other primary views

Safety boundaries

  • dynamic HTML, API/JSON responses, ranges, binary assets, HEAD bodies, and non-GET/HEAD cache paths are never compressed or marked immutable
  • static responses that mutate an authentication cookie are forced to private, no-store
  • gzip and identity representations use distinct ETags, while CSP and the existing security headers remain intact
  • gzip-aware HEAD responses expose the same encoding, length, validator, variance, and cache policy as equivalent GET responses while retaining an empty body
  • static conditional validators are evaluated after content-encoding negotiation, so an identity ETag cannot validate an unacceptable gzip-only request
  • cache policy is classified from the final post-precondition status, keeping failures such as 412 out of shared caches
  • the server grants immutable caching only when the URL hash matches the current file content; malformed, mismatched, ambiguous, and missing query URLs are no-store, while clean unversioned URLs must revalidate
  • hidden diagnostics canvases are not redrawn from their device-pixel backing dimensions; CSS retains control of logical sizing and charts redraw after Workspaces becomes visible again
  • no runtime dependency or lock-file changes are required

Performance evidence

Measured against 615ede3 with Chromium, cold browser cache, 5 Mbit/s throughput, 80 ms latency, and seven alternating runs:

Metric Before After Change
Median login load 4,181 ms 572 ms -86.3%
Transferred by load 2,630,671 B 226,067 B -91.4%
Median DOMContentLoaded 2,255 ms 321 ms -85.8%
First Contentful Paint 992 ms 344 ms -65.3%

Five alternating warm-cache runs reduced median login load from 273 ms to 116 ms (-57.6%) and transferred bytes from 20,609 B to 11,877 B (-42.4%). The authentication translation bundle is 76,308 B instead of 521,884 B (-85.4% raw).

Testing

  • pytest -q: 2,352 passed, 29 skipped
  • static delivery regression suite: 13 passed, including gzip GET/HEAD metadata parity
  • JavaScript unit suite: 38 passed
  • Playwright E2E shards: 104 passed
  • complete session workspace browser suite: 16 passed, including four repeated Diagnostics/File Manager navigation cycles
  • post-hardening auth/load, layout, and theme browser regression: 14 passed
  • authentication bundle freshness check, ESLint, vendored asset verification, and git diff --check

Fixes #192

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 2, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-02T21:50:11.702724Z 16641ab Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4ff89c61d3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/static_delivery.py Outdated
Comment thread app/static_delivery.py Fixed
Comment thread app/static_delivery.py Fixed
Comment thread app/static_delivery.py Fixed
Comment thread app/static_delivery.py Fixed
@bifrost0x

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Can't wait for the next one!

Reviewed commit: 859cfa903e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@bifrost0x bifrost0x changed the title Optimize initial page load and static asset delivery Optimize page loads and stabilize diagnostics charts Sep 2, 2026
@bifrost0x

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 951951a219

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/static_delivery.py Outdated
@bifrost0x

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 127d2123c9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/static_delivery.py
@bifrost0x

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0a740ec97e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/static_delivery.py Outdated
@bifrost0x

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: 16641ab342

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@bifrost0x
bifrost0x merged commit fecaa86 into main Sep 2, 2026
20 of 21 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in WebSSH Roadmap Sep 2, 2026
@bifrost0x
bifrost0x deleted the dev/optimize-initial-page-load branch September 2, 2026 21:52
@bifrost0x bifrost0x self-assigned this Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Diagnostics pane graphs getting bigger and bigger

2 participants